Skip to content

fix: confirm before discarding an incomplete recording - #2459

Merged
richiemcilroy merged 1 commit into
CapSoftware:mainfrom
AatmanAJ:fix/confirm-discard-incomplete-recording
Oct 13, 2026
Merged

richiemcilroy merged 1 commit into
CapSoftware:mainfrom
AatmanAJ:fix/confirm-discard-incomplete-recording

Conversation

@AatmanAJ

@AatmanAJ AatmanAJ commented Oct 10, 2026 •

Copy link
Copy Markdown
Contributor

Fixes #2365

In the Tauri app (apps/desktop), the recovery toast's Discard button deleted the whole recording folder as soon as it was clicked. In #2365, that's how a 38-minute recording was lost after recovery kept failing. That recovery failure is #2386, fixed in #2458.

Discard now asks first, using the prompt the library's delete button already uses: "Are you sure you want to delete this recording?" The GPUI app already shows this same confirmation before its Discard (MainWindow::process_incomplete_recording), so the shipping app needs no change.

Testing

  • Biome and tsc -p apps/desktop --noEmit pass.
  • dialog:default is granted to every window (capabilities/default.json), so ask() works in the main window where the toast renders.
  • Hand-tested in bun run dev:desktop on macOS 27.0.1:
    • With an incomplete recording listed, Discard shows "Are you sure you want to delete this recording?"
    • No keeps the recording.
    • Yes deletes it, and logs a single Discarded incomplete recording line, at the moment of the Yes.
    • To get an incomplete recording, I force-killed the app during a Studio recording and set the recording's status to NeedsRemux. A force-killed InProgress Studio recording is marked Failed: "Recording crashed" at the next launch (resume_uploads), and the recovery toast skips Failed recordings, so it never offers Discard for one. That is existing behaviour, unrelated to this change.

RetriggerConfidence Score: 5/5

The PR appears safe to merge; no actionable issues were found.

Summary

RecoveryToast.tsx now asks before discarding an incomplete recording.

  • The recovery toast asks before discarding an incomplete recording.

No actionable issues found. Review was based on source inspection; no runtime or visual checks were performed.

Reviews (1) · Last reviewed commit: "fix(desktop): confirm before discarding ..." · Reviewed by Greptile

The recovery toast's Discard button deleted the whole recording folder
straight away. Ask first, with the same prompt the library's delete
button and the GPUI app's Discard already use.

Fixes CapSoftware#2365
@AatmanAJ
AatmanAJ marked this pull request as ready for review October 11, 2026 09:03
@richiemcilroy
richiemcilroy merged commit 8a14802 into CapSoftware:main Oct 13, 2026
18 of 19 checks passed
@richiemcilroy

Copy link
Copy Markdown
Member

Thanks for the PR :)

@MaximAndreevich

Copy link
Copy Markdown

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Macos: Operations that result in irreversible data loss must require a confirmation dialog.

3 participants